Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor dummy scorables. #14046

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Conversation

jpountz
Copy link
Contributor

@jpountz jpountz commented Dec 6, 2024

This makes our scorers that produce dummy scorables share the same implementation.

This makes our scores that produce dummy scorables share the same implementation.
@jpountz jpountz added this to the 10.1.0 milestone Dec 6, 2024
Copy link
Member

@benwtrent benwtrent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love seeing deleted code 😍

@jpountz jpountz merged commit a833887 into apache:main Dec 9, 2024
3 checks passed
@jpountz jpountz deleted the refactor_dummy_scorable branch December 9, 2024 14:39
jpountz added a commit that referenced this pull request Dec 9, 2024
This makes our scores that produce dummy scorables share the same implementation.
@jpountz
Copy link
Contributor Author

jpountz commented Dec 10, 2024

There was a regression on nightly benchmarks last night (https://benchmarks.mikemccandless.com/2024.12.09.18.05.31.html) that is either due to this PR or to newly added tasks (CountOrMany, CountFilteredOrMany). I'm reverting this PR to figure out which one is the culplit. I'll add it back if performance doesn't improve.

jpountz added a commit that referenced this pull request Dec 10, 2024
jpountz added a commit that referenced this pull request Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants